Skip to content
This repository was archived by the owner on Sep 22, 2025. It is now read-only.

Fixed the blank spaces appearing on the exercise portal and made the page responsive.#335

Open
KaustubhK98 wants to merge 6 commits intomasterfrom
gm-frontend-kaustubh
Open

Fixed the blank spaces appearing on the exercise portal and made the page responsive.#335
KaustubhK98 wants to merge 6 commits intomasterfrom
gm-frontend-kaustubh

Conversation

@KaustubhK98
Copy link
Collaborator

@KaustubhK98 KaustubhK98 commented Aug 10, 2025

Description

The exercise module was using a "Grid Layout" to handle the exercise card actions. I replaced the "Grid Layout" with a "Recycler View". Now, the exercise cards will be dynamically added or removed from the adapter and after every action the page will be automatically refreshed. This issue has fixed the bug, which lead to blank spaces on the page and made the UI more responsive. Recycler View is a future proof solution over Grid Layout due to its ability to dynamically handle card actions and scalability.

Todos

  • Tested and working locally
  • Code follows the style guidelines of this project
  • I have performed a self-review of my code
  • Code changes documented
  • Requested review from >= 2 devs on the team (one frontend and one backend recommended)

How to test

  1. Login to app
  2. Go to exercise module page
  3. Select any module and click on "mark as complete" or "save for later" button.
  4. The card should be gone and there shouldn't be any blank space on the page.
  5. The UI is dynamic now and is updated automatically after any changes are made to the exercise cards.

Screenshots and/or Gifs

Uploading Screen Recording 2025-08-10 at 9.24.52 pm.mov…

Associated MS Planner Tasks

Known Issues

There aren't any known issues arising due to this change.

@AryanSharma1017
Copy link
Collaborator

AryanSharma1017 commented Aug 27, 2025 •
edited
Loading

Hey Kaustubh, just check for the build once again by running ./gradlew spotlessapply and then check for the ./gradlew build and then see, I think then all these errors should go.
if not then please have a talk with me and we will see.

@KaustubhK98 KaustubhK98 force-pushed the gm-frontend-kaustubh branch from 5f9a416 to f10fd62 Compare August 29, 2025 09:45
@KaustubhK98
Copy link
Collaborator Author

KaustubhK98 commented Aug 29, 2025

Hi Aryan,

Spotless build wasn't an issue. There was a Gradle version mismatch, I have fixed the problem on my end and raised another PR. Can you please let me know if you are able to see the new PR. Please try to merge the code now, I am certain that it will get merged this time. Please let me know if there are any problems while merging the new PR.

Thanks!

@KaustubhK98
Night mode removed -Kaustubh
c290e5f
@KaustubhK98
Fixed gradle spotless related issues -Kaustubh
c6291fa
@KaustubhK98
Update ApiClient base URL
66cc09f
@KaustubhK98
Added a functionality to assign a doctor to a patient and send a requ…
b55ba17 
…est to the doctor. Also, added a check to prevent sending duplicate requests. Added a search bar to search for doctors from the list
@KaustubhK98
Copy link
Collaborator Author

KaustubhK98 commented Sep 21, 2025

1. Added functionality to assign doctor to a patient from the caretaker's profile:

  • A caretaker can now send a request to a doctor before actually assigning a patient to them.
  • A caretaker can directly assign a patient to the doctor
  • A caretaker can Un-assign or cancel a request sent to a doctor.
  • The app doesn’t allow duplicate requests
  • Doctors can be searched using the search bar.

2. Safely removed the night mode and it's files from the app.

3. Added a function to view the list of patients assigned to a doctor. This function is partially implemented and needs the doctor's portal to work properly.

Known Issues
There aren't any known issues arising due to this change.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@KaustubhK98 @AryanSharma1017